-
-
Notifications
You must be signed in to change notification settings - Fork 11
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add GitHub Actions #20
Conversation
Log workspace name and assert that the config should not be null
Split `getWorkspaceConfig()` and `parseWorkspaceConfig()` so the latter one can be tested simply by passing in a path to the `.remarkrc` file we want to test.
Move the `test` directory underneath the `test` directory in `src` and rename it to `json`.
7821b88
to
b749c0d
Compare
@mrmlnc, this is now rebased against the |
I gave you the rights to manage the repository (includes merge of PR's). |
Excellent, thanks, @mrmlnc. If you want me to follow up on remarkjs/remark#508 (comment), perhaps you can make me an admin of the repository? Seeing how I'm going to have that after the repository is moved anyway, I might as well have it now, no? |
This PR adds a GitHub Actions workflow that performs extensive tests on the parsing of
.remarkrc
in preparation for a fix for #15. Without a continuous integration system with automatic tests, it's hard to debug and fix problems. This PR tries to remedy that.The PR also upgrades the entire extension to be on par with the current Yeoman template, which was the only way I was able to get the automatic tests to work.
I can also warmly recommend that you set up Dependabot to keep all dependencies in this extensions up to date. It's free and creates pull requests with changes to
package.json
andpackage-lock.json
whenever a dependency has a new version. It's also highly configurable so you can ignore minor versions, major versions, whole packages, etc.